Guide Box text and preview retrieval fallbacks#259
Open
william-openai wants to merge 3 commits into
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
get_file_contentand Deep Researchfetchreadssizeinto preview selection soget_file_previewis avoided for files already known to exceed Box's 3 MB limitWhy
The Box invalid-argument review for UTC May 14-20 found
get_file_contenttext-representation misses as a clean skill-guidance target: 490 errors, or 13.5% of Box errors in that window. The residual-tail pass also found the same explicit missing-representation message in 37 Boxfetcherror-log rows; this PR folds that exact subset into the same text-read guidance.A separate residual preview pass found the 3 MB preview limit dominates residual
get_file_previewinvalid arguments: 40 size-cap log shapes among 53 residual preview errors.The broader
fetchbucket remains mixed: Box also returns one error shape that conflates not-found, non-file, and non-extractable-text cases. This PR does not claim to fix that mixed shape. Generic Box AI invalids, metadata schema lookup, and metadata search also did not justify more skill text because they were already described by the MCP surface/provider error or opaque without more Box-side argument validation detail.Validation
git diff --check -- plugins/boxnode plugins/plugin-eval/scripts/plugin-eval.js analyze plugins/box --format markdown(reports existing Box plugin budget/structure warnings; no failures)